Skip to content

chore!: update engines.node to minimum of 20 #9128

New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Merged
merged 4 commits into from
Jul 15, 2025
Merged

chore!: update engines.node to minimum of 20 #9128

merged 4 commits into from
Jul 15, 2025

Conversation

hsubox76
Copy link
Contributor

@hsubox76 hsubox76 commented Jun 26, 2025

Node 18 is now no longer maintained by Node and is deprecated by Google Cloud as well https://cloud.google.com/functions/docs/runtime-support#node.js

Node 20 is in "maintenance" and should be a safe minimum requirement. https://nodejs.org/en/about/previous-releases

Copy link

changeset-bot bot commented Jun 26, 2025

🦋 Changeset detected

Latest commit: dc4e9ba

The changes in this PR will be included in the next version bump.

This PR includes changesets to release 32 packages
Name Type
@firebase/rules-unit-testing Major
@firebase/app-check-compat Minor
@firebase/firestore-compat Minor
@firebase/functions-compat Minor
@firebase/database-compat Minor
@firebase/storage-compat Minor
@firebase/auth-compat Minor
@firebase/app-compat Minor
@firebase/app-check Minor
@firebase/component Minor
@firebase/firestore Minor
@firebase/functions Minor
@firebase/database Minor
@firebase/storage Minor
@firebase/logger Minor
@firebase/auth Minor
@firebase/util Minor
@firebase/app Minor
@firebase/ai Minor
firebase Major
@firebase/analytics-compat Patch
@firebase/installations-compat Patch
@firebase/messaging-compat Patch
@firebase/performance-compat Patch
@firebase/remote-config-compat Patch
@firebase/analytics Patch
@firebase/data-connect Patch
@firebase/installations Patch
@firebase/messaging Patch
@firebase/performance Patch
@firebase/remote-config Patch
@firebase/database-types Patch

Not sure what this means? Click here to learn what changesets are.

Click here if you're a maintainer who wants to add another changeset to this PR

@google-oss-bot
Copy link
Contributor

google-oss-bot commented Jun 26, 2025

@google-oss-bot
Copy link
Contributor

google-oss-bot commented Jun 26, 2025

@hsubox76 hsubox76 changed the title chore: update engines.node to minimum of 22 chore: update engines.node to minimum of 20 Jun 30, 2025
@hsubox76 hsubox76 added this to the v12 milestone Jul 8, 2025
@hsubox76 hsubox76 marked this pull request as ready for review July 8, 2025 18:16
@dlarocque
Copy link
Contributor

Can you add a changeset and indicate that this is a breaking change in the title (!)

@hsubox76 hsubox76 changed the title chore: update engines.node to minimum of 20 chore!: update engines.node to minimum of 20 Jul 9, 2025
@hsubox76 hsubox76 requested a review from a team as a code owner July 9, 2025 22:25
@hsubox76
Copy link
Contributor Author

hsubox76 commented Jul 9, 2025

Can you add a changeset and indicate that this is a breaking change in the title (!)

Added, "chore!" looks really weird, maybe that's not the right category. Doesn't affect the build though.

@hsubox76 hsubox76 merged commit 25b60fd into main Jul 15, 2025
120 of 129 checks passed
@hsubox76 hsubox76 deleted the ch-node-minimum branch July 15, 2025 16:48
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

5 participants